Skip to content

Return an SVector in piessens#136

Merged
hyrodium merged 3 commits intoJuliaApproximation:masterfrom
jishnub:jishnub/piessens_alloc
Oct 19, 2025
Merged

Return an SVector in piessens#136
hyrodium merged 3 commits intoJuliaApproximation:masterfrom
jishnub:jishnub/piessens_alloc

Conversation

@jishnub
Copy link
Member

@jishnub jishnub commented May 13, 2025 •
edited
Loading

This should reduce some allocations. After this,

(FastGaussQuadrature) julia> piessens(0.0)
6-element SVector{6, Float64} with indices SOneTo(6):
  2.40482555769751
  5.520078110288018
  8.653727912912164
 11.791534439013164
 14.930917708486993
 18.071063967911865

@codecov
Copy link

codecov bot commented Oct 19, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.88%. Comparing base (49a6217) to head (e3f4d8a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   96.89%   96.88%   -0.01%     
==========================================
  Files           9        9              
  Lines        1384     1382       -2     
==========================================
- Hits         1341     1339       -2     
  Misses         43       43

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

hyrodium
hyrodium approved these changes Oct 19, 2025
View reviewed changes
Copy link
Collaborator

@hyrodium hyrodium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR!

@hyrodium hyrodium merged commit 6b19071 into JuliaApproximation:master Oct 19, 2025
9 checks passed
@jishnub jishnub deleted the jishnub/piessens_alloc branch October 19, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hyrodium hyrodium hyrodium approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jishnub @hyrodium